fix(edit-content): date pickers use fixed width and stay open until click-outside (#36156)#36365
Merged
adrianjm-dotCMS merged 1 commit intoJun 30, 2026
Conversation
…lick-outside (#36156) Date, Date/Time and Time field pickers in the new Edit Contentlet were stretching to full width and closing as soon as a date was clicked, preventing users from picking the time portion. - Remove the w-full / inputStyleClass overrides and force-width SCSS so the picker uses PrimeNG's default datepicker sizing instead of full width. - Set [hideOnDateTimeSelect]="false" so the panel stays open after a date selection and only closes on click-outside, consistent across all three field types. The value is still applied on (onSelect). - Add spec coverage for both behaviors across Date, Date/Time and Time. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
9 tasks
github-actions
Bot
added
the
Area : Frontend
Contributor
|
Claude finished @adrianjm-dotCMS's task in 1m 8s —— View job Rollback Safety Analysis
Result: ✅ Safe To Rollback All three changed files are purely frontend (Angular component template, SCSS, and spec):
Category check:
No backend, database, Elasticsearch, API, or OSGi changes are present. This is a UI-only fix affecting PrimeNG datepicker width and close behavior. Rolling back to the previous release requires no database interaction, no reindex, and no manual ops action. Label |
Contributor
🤖 dotBot Review (Bedrock)Reviewed 3 file(s); 0 candidate(s) → 0 confirmed, 0 uncertain (unverified, kept for review). ✅ No issues found after verification. us.deepseek.r1-v1:0 · Run: #28451631239 · tokens: in: 7275 · out: 1127 · total: 8402 · calls: 3 · est. ~$0.016 |
adrianjm-dotCMS
requested review from
KevinDavilaDotCMS,
dario-daza,
nicobytes,
oidacra,
rjvelazco and
zJaaal
zJaaal
approved these changes
Jun 30, 2026
|
Tick the box to add this pull request to the merge queue (same as
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed Changes
Fixes the Date, Date/Time and Time field pickers in the new Edit Contentlet (issue #36156):
w-fullclass,[inputStyleClass]="'w-full'"binding and thewidth: 100%SCSS override. The picker now uses PrimeNG's default datepicker sizing (display: inline-flex, hugging the input + dropdown button) instead of stretching full width. No hard-coded magic number.[hideOnDateTimeSelect]="false"so the panel stays open after a date is clicked and only closes on click-outside. This lets users complete the time portion on Date/Time and Time fields. Consistent across all three field types. The selected value is still applied on(onSelect), so it is correctly persisted when the picker closes.Screen.Recording.2026-06-30.at.10.43.45.AM.mov
Checklist
Additional Info
Single-component change: all three field types are rendered by
dot-calendar-field, so the fix applies to Date, Date/Time and Time automatically. No API/backend changes.Verification:
pnpm nx test edit-content→ 1871 passed (includes 6 new cases)pnpm nx lint edit-content→ 0 errorsScreenshots
Closes #36156
This PR fixes: #36156